Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cherrypick ImageAugmetation related change to R2.9 #16454

Merged
merged 10 commits into from
Apr 22, 2022
Merged

Conversation

qlzh727
Copy link
Member

@qlzh727 qlzh727 commented Apr 22, 2022

Piper cl: 443122383

qlzh727 and others added 10 commits April 7, 2022 15:55
PiperOrigin-RevId: 438858608
This is one test file failing, due to the monkey patching happens in the dtensor.init(), and I will need to dig more about the root cause (probably due to patching tf.Variable with DVariable, and cause logic difference for instance type checking.)

PiperOrigin-RevId: 439676157
1. Add docstring for LayoutMap.
2. Hide certain methods for keras.dtensor.optimizers.

PiperOrigin-RevId: 442651549
PiperOrigin-RevId: 443122383
```
  def augment_label(self, label, transformation=None):
    return label
```
on remaining image augmentation layers

PiperOrigin-RevId: 443446864
@google-ml-butler google-ml-butler bot added the keras-team-review-pending Pending review by a Keras team member. label Apr 22, 2022
@qlzh727 qlzh727 removed the keras-team-review-pending Pending review by a Keras team member. label Apr 22, 2022
@qlzh727
Copy link
Member Author

qlzh727 commented Apr 22, 2022

The presubmit failure is expected since the test was configured with run with tf-nightly, whereas it should run with tf 2.9 rc. The same test has been run when we merge the original change.

We will run the same test again with tf 2.9 rc when we bake the keras 2.9 rc binary.

@qlzh727 qlzh727 merged commit 07e1374 into keras-team:r2.9 Apr 22, 2022
@qlzh727 qlzh727 deleted the r2.9 branch November 8, 2022 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants